Skip to content

Conversation

mariehd
Copy link
Member

@mariehd mariehd commented Sep 7, 2026

Endringer

  • La til useState for å holde styr på hvilken sang som vises
  • Koblet valgt sang til SongCard
  • Implementerte fungerende forrige- og neste-knapper
  • La til callbacks for navigering i Navigation-komponenten
  • Implementerte valg av sang fra SongList
  • Når brukeren klikker på en sang i oversikten, oppdateres sangen som vises
  • Navigering med forrige/neste fortsetter fra sangen som er valgt

Test

  • npm run lint fungerer uten feil
  • npm run build fungerer uten feil
  • Testet at neste-knappen viser neste sang
  • Åpne appen med nam run devog:
    • Test at forrige-knappen viser forrige sang
    • Test at knappene deaktiveres ved første og siste sang
    • Testet at valg av sang fra oversikten oppdaterer SongCard
    • Testet at forrige/neste fungerer etter at en sang er valgt fra oversikten

Closes #24 og #28

@mariehd mariehd requested a review from a team as a code owner September 7, 2026 10:02
@mariehd mariehd marked this pull request as draft September 8, 2026 12:40
@mariehd mariehd changed the title Implementert bla frem og tilbake Implementert navigasjon og bla mellom sanger Sep 8, 2026
@mariehd mariehd marked this pull request as ready for review September 8, 2026 12:50
Copy link
Member

@ellahw ellahw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ser bra ut! Endret bare rekkefølgen på noe av koden i App() hvor jeg flyttet useState-linjene lenger opp.

@ellahw ellahw merged commit 1f266c7 into main Sep 8, 2026
@ellahw ellahw deleted the 24-implementer-fremtilbake-state branch September 8, 2026 13:26
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implementer "hopp til sang" (state) Implementer bla frem/tilbake (state)
2 participants